Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use new CircleCI convenience image for PHP #177

Merged
merged 17 commits into from
Mar 11, 2022

Conversation

byrond
Copy link
Contributor

@byrond byrond commented Nov 29, 2021

See https://discuss.circleci.com/t/legacy-convenience-image-deprecation/41034

Todo:

  • Update defaults/install/.circleci/config.yml
  • Maybe add a comment with a link to the orb documentation for installing browsers?
  • Maybe add a comment about when the mysql image will get a cimg version? Update for new MySQL image.
  • The devel version issue should be resolved in the develop branch now

@byrond byrond self-assigned this Nov 29, 2021
@becw
Copy link
Member

becw commented Dec 2, 2021

I added some todos to the description but this change sounds good!

@byrond
Copy link
Contributor Author

byrond commented Jan 4, 2022

@becw, even though #176 is merged, Composer grabs the latest release of The Build (which is 4.0-alpha4) during install. That version doesn't have the Devel fix, so palantirnet/drupal-skeleton#119 will fail until there is a new release of 4.x of The Build.

[Edited to correct the installed version in the skeleton and clarify that the release doesn't need to be stable.]

@byrond
Copy link
Contributor Author

byrond commented Jan 4, 2022

@becw, I created https://github.com/palantirnet/the-build/releases/tag/4.0-alpha5 and palantirnet/drupal-skeleton#119 is passing now without specifying the dev branch.

.circleci/config.yml Outdated Show resolved Hide resolved
@byrond byrond requested a review from becw January 14, 2022 20:18
@byrond byrond marked this pull request as ready for review January 14, 2022 20:18
@byrond
Copy link
Contributor Author

byrond commented Jan 20, 2022

@becw I just made one last update to the example deployment config snippets. Otherwise, I think this is ready to go.

@becw becw requested a review from eric-schmidt January 20, 2022 16:50
@becw becw merged commit 463a3f1 into develop Mar 11, 2022
@becw becw deleted the circleci-convenience-image-update branch March 11, 2022 00:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants